Adding "perf mem" tests #2786
Adding "perf mem" tests #2786
Conversation
|
Test results: [root@ltcden4-lp2-test perf]# avocado run perf_mem.py -m perf_mem.py.data/record_report.yaml --max-parallel-tasks=1 Test run without YAML file[root@ltcden4-lp2-test perf]# avocado run perf_mem.py --max-parallel-tasks=1 |
|
Test result directory |
| input_data: | ||
| record_method: -i -e ldlat-stores | ||
| input_data_full: | ||
| record_method: --input -e ldlat-stores |
There was a problem hiding this comment.
we can remove these lines as -i is used with perf report
| event_store_load: | ||
| record_method: -e ldlat-stores -t load -f -i -v -C 0 |
There was a problem hiding this comment.
same here also, can change it to
record_method: -e ldlat-stores -t load -f -v -C 0
perf/perf_mem.py
Outdated
| # file arguments for record command | ||
| if self.record == "-i -e ldlat-stores": | ||
| self.record = "-i %s -e ldlat-stores" % output_file | ||
| elif self.record == "--input -e ldlat-stores": | ||
| self.record = "--input=%s -e ldlat-stores" % output_file | ||
| elif self.record == "-e ldlat-stores -t load -f -i -v -C 0": | ||
| self.record = "-e ldlat-stores -t load -f \ | ||
| -i %s -v -C 0" % output_file | ||
|
|
There was a problem hiding this comment.
no need of this code after removing -i from yaml with record command
perf/perf_mem.py
Outdated
| import platform | ||
| import tempfile |
There was a problem hiding this comment.
unused imports can be removed
Adding tests of "perf mem" command with main options called "record" and "report" with all possible options through YAML file. Signed-off-by: Shaik Abdulla <[email protected]>
FarooqAbdulla02
force-pushed
the
perf_mem
branch
from
7ffe140
to
ffc767c
Compare
PraveenPenguin
merged commit 3ee8f7a
into
avocado-framework-tests:master
|
Code looks good to me. |
Adding tests of "perf mem" command with main options called
"record" and "report" with all possible options through YAML file.